Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Run all program-test in SBF mode and speed up CI #20

Merged
merged 7 commits into from
Feb 8, 2024
Merged

Conversation

buffalu
Copy link
Collaborator

@buffalu buffalu commented Feb 8, 2024

  • Run in SBF mode to make sure actual BPF and runtime restrictions are being enforced. This should help us catch runtime errors before they make it to prod.
  • Speed up CI to get fast iteration cycles
  • Cleanup some compiler warnings by renaming functions to not have overlap
Screenshot 2024-02-07 at 11 48 27 PM

Speed up CI a bunch
Some cleanup for compiler warnings
@buffalu buffalu requested a review from ebatsell February 8, 2024 05:47
@buffalu buffalu changed the title Run all program-test in SBF mode Run all program-test in SBF mode and speed up CI Feb 8, 2024
Copy link
Collaborator

@ebatsell ebatsell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

so what's the new command to run tests? the test script you had earlier is missing

uses: baptiste0928/cargo-install@v3
with:
crate: anchor-cli
version: "0.29.0"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this should be anchor 0.28

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CI not liking 0.28.0. unless you feel strongly about it, 0.29.0 seems fine as 0.28.0 never made it to crates.io

@@ -797,7 +797,7 @@
"type": "u8"
},
{
"name": "num_addrs",
"name": "numAddrs",
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this shouldn't change as long as we're still on anchor 0.28, maybe the solana program test compatibility is fixed with the new way we're loading in programs but we'd still need to bump versions and make the code compatible

@buffalu
Copy link
Collaborator Author

buffalu commented Feb 8, 2024

so what's the new command to run tests? the test script you had earlier is missing

added to readme

@buffalu buffalu merged commit 2b59311 into master Feb 8, 2024
6 checks passed
@buffalu buffalu deleted the lb/fixture branch February 8, 2024 22:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants